-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: unmanaged dependency check #2223
Conversation
Todo:
Until the design doc is approved, leave this pull request as failed state in draft. To recover from the failure: Assuming the (unmanaged) dependency went through the governance process,
(Before merging this 2223, we'll revert the edits in the shared dependencies BOM and gax-java/gax-grpc/pom.xml.) |
In order to fail the check, an unmanaged dependency has to be added in the target bom, not the dependency of this bom. The check needs to update. I added a dependency in
When tested locally:
|
...cies/unmanaged-dependency-check/src/main/java/com/google/cloud/UnmanagedDependencyCheck.java
Show resolved
Hide resolved
...cies/unmanaged-dependency-check/src/main/java/com/google/cloud/UnmanagedDependencyCheck.java
Outdated
Show resolved
Hide resolved
...cies/unmanaged-dependency-check/src/main/java/com/google/cloud/UnmanagedDependencyCheck.java
Show resolved
Hide resolved
...cies/unmanaged-dependency-check/src/main/java/com/google/cloud/UnmanagedDependencyCheck.java
Outdated
Show resolved
Hide resolved
...cies/unmanaged-dependency-check/src/main/java/com/google/cloud/UnmanagedDependencyCheck.java
Show resolved
Hide resolved
This reverts commit 38b3ee4.
Quality Gate passed for 'gapic-generator-java-root'Kudos, no new issues were introduced! 0 New issues |
Quality Gate passed for 'java_showcase_integration_tests'Kudos, no new issues were introduced! 0 New issues |
Implement an unmanaged dependency check, part of dependency governance project.